Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TECH] add direct metrics pushed to datadog #10631

Open
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

Mefl
Copy link
Contributor

@Mefl Mefl commented Nov 25, 2024

🍂 Problème

Les dashboards DD sont nourris à partir des logs et donc nous sommes amnésiques après quinze jours.

🌰 Proposition

Mettre en place des metrics et les publier directement depuis l'application.

🎃 Remarques

2 FT sont ajoutés afin de permettre une transistion smooth vers la prod : on peut activer ou pas les métriques et désactiver ou pas la collecte des métriques knex via oppsy.

🪵 Pour tester

Déployer en RA et voir la bonne réception des métriques dans DD

@Mefl Mefl added the POC Pour les PoC label Nov 25, 2024
@Mefl Mefl requested review from MathieuGilet and AnaisAllamand and removed request for AnaisAllamand November 25, 2024 15:40
@pix-bot-github
Copy link

Une fois les applications déployées, elles seront accessibles via les liens suivants :

Les variables d'environnement seront accessibles via les liens suivants :

@Mefl Mefl changed the title add direct metrics pushed to datadog [TECH] add direct metrics pushed to datadog Nov 25, 2024
@Mefl Mefl force-pushed the tech-add-metrics-on-routes branch 4 times, most recently from 6a9db5e to e382aaf Compare November 26, 2024 15:12
@Mefl Mefl force-pushed the tech-add-metrics-on-routes branch 7 times, most recently from 217a7e1 to 173e237 Compare December 17, 2024 16:11
@Mefl Mefl force-pushed the tech-add-metrics-on-routes branch from 173e237 to ab47875 Compare December 18, 2024 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
POC Pour les PoC
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants